Skip to content

Sxxx - some adjustments#2220

Open
mariusheine wants to merge 8 commits intomainfrom
sxxx-adjustments
Open

Sxxx - some adjustments#2220
mariusheine wants to merge 8 commits intomainfrom
sxxx-adjustments

Conversation

@mariusheine
Copy link
Contributor

@mariusheine mariusheine commented Mar 4, 2026

No description provided.

@github-actions
Copy link

github-actions bot commented Mar 4, 2026 •
edited
Loading

Test Results

131 tests  ±0   131 ✅ ±0   26s ⏱️ +4s
 31 suites ±0     0 💤 ±0 
 31 files   ±0     0 ❌ ±0 

Results for commit e666bb0. ± Comparison against base commit 3674834.

This pull request removes 9 and adds 9 tests. Note that renamed tests count towards both. 
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[1] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@1dc3502b
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[2] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@59696551
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[3] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@15c4af7a
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[4] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@55d58825
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[5] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@29a98d9f
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKey(TestData)[1] TestData [input=org.eclipse.set.model.planpro.Weichen_und_Gleissperren.impl.W_Kr_Gsp_ElementImpl@795faad, expected=multi/container-id/2025652936/element-id]
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKey(TestData)[2] TestData [input=org.eclipse.set.model.planpro.Signale.impl.SignalImpl@69f3e556, expected=multi/456/1206258545/123]
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKeyOther(TestData)[1] TestData [input=Pair [first=org.eclipse.set.model.planpro.Weichen_und_Gleissperren.impl.W_Kr_Gsp_ElementImpl@99c3cee, second=other], expected=multi/container-id/1482010861/element-id/other=other]
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKeyOther(TestData)[2] TestData [input=Pair [first=org.eclipse.set.model.planpro.Signale.impl.SignalImpl@60ab895f, second=xxx], expected=multi/456/1097632693/123/other=xxx]

org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[1] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@5e5af8e1
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[2] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@4228bf58
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[3] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@2938127d
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[4] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@ff23ae7
org.eclipse.set.basis.graph.DigraphsTest ‑ testGetPathsDigraphPP(TestGetPathsDigraphPP)[5] org.eclipse.set.basis.graph.DigraphsTest$TestGetPathsDigraphPP@2b7962a2
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKey(TestData)[1] TestData [input=org.eclipse.set.model.planpro.Weichen_und_Gleissperren.impl.W_Kr_Gsp_ElementImpl@69d7bc1a, expected=multi/container-id/1170475172/element-id]
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKey(TestData)[2] TestData [input=org.eclipse.set.model.planpro.Signale.impl.SignalImpl@6e668322, expected=multi/456/88332821/123]
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKeyOther(TestData)[1] TestData [input=Pair [first=org.eclipse.set.model.planpro.Weichen_und_Gleissperren.impl.W_Kr_Gsp_ElementImpl@6475e778, second=other], expected=multi/container-id/1841931630/element-id/other=other]
org.eclipse.set.ppmodel.extensions.utils.CacheUtilsTest ‑ testGetCacheKeyOther(TestData)[2] TestData [input=Pair [first=org.eclipse.set.model.planpro.Signale.impl.SignalImpl@acd3460, second=xxx], expected=multi/456/82436294/123/other=xxx]

♻️ This comment has been updated with latest results.

@github-actions github-actions bot mentioned this pull request Mar 4, 2026
Closed
eclipse-set-bot and others added 2 commits March 9, 2026 12:13
@eclipse-set-bot
Update table reference
2548f3a
@mariusheine @TruongQuangSB
fix table transformation for missing tables
e666bb0 
Co-authored-by: TruongQuangSB <131350493+TruongQuangSB@users.noreply.github.com>
@TruongQuangSB
Copy link
Contributor

TruongQuangSB commented Mar 9, 2026 •
edited
Loading

@mariusheine I think, the width of "calculate missing table panel" be equal the table width or fill whole view

@mariusheine
Copy link
Contributor Author

mariusheine commented Mar 9, 2026

@mariusheine I think, the width of "calculate missing table panel" be equal the table width or fill whole view

@TruongQuangSB I think then it is nicer to fill the whole view. And you mean that the yellow background should take the full width and that the button is on the right side, correct?

@TruongQuangSB
Copy link
Contributor

TruongQuangSB commented Mar 9, 2026 •
edited
Loading

@mariusheine I think, the width of "calculate missing table panel" be equal the table width or fill whole view

@TruongQuangSB I think then it is nicer to fill the whole view. And you mean that the yellow background should take the full width and that the button is on the right side, correct?

@mariusheine yes. You can see the Banderole.java, here haved we the top-panel for the view already defined. For this fall need we only override the labe & the buttion action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TruongQuangSB TruongQuangSB TruongQuangSB left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mariusheine @TruongQuangSB @eclipse-set-bot